Skip to content

Update CI workflows for self-hosted runners#1627

Merged
gemcoder21 merged 20 commits intomainfrom
ci-runner
Jan 24, 2026
Merged

Update CI workflows for self-hosted runners#1627
gemcoder21 merged 20 commits intomainfrom
ci-runner

Conversation

@gemcoder21
Copy link
Contributor

Changed the 'runs-on' property to use self-hosted macOS runners in both CI and UI test workflows. Added conditional execution for caching and sccache steps to only run on GitHub-hosted environments. Removed unnecessary sccache stats verification from UI tests.

Changed the 'runs-on' property to use self-hosted macOS runners in both CI and UI test workflows. Added conditional execution for caching and sccache steps to only run on GitHub-hosted environments. Removed unnecessary sccache stats verification from UI tests.
@gemini-code-assist
Copy link
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

Eliminated the 'Select Xcode' step from both ci.yml and ui_tests.yml GitHub Actions workflows as it is no longer necessary.
Moves SCCACHE_GHA_ENABLED environment variable setup into a conditional step for GitHub-hosted runners in both ci.yml and ui_tests.yml. This ensures the variable is only set when appropriate, improving workflow flexibility.
Added a conditional to only install 'just' with Homebrew when the workflow is running on GitHub-hosted runners in both ci.yml and ui_tests.yml. This prevents unnecessary installation steps on self-hosted or other environments.
Added commented lines for 'runs-on: macos-26' in both ci.yml and ui_tests.yml to document the previous runner configuration while continuing to use self-hosted macOS runners.
The CI workflow now explicitly cleans DerivedData, builds the project, and runs tests without building. Added a new 'test-without-building' recipe to the justfile and updated the workflow to use it, improving build efficiency and clarity.
Consolidated test-all and test-without-building into a single _test action recipe to reduce duplication. Updated test-all and test-without-building to use the new _test action with appropriate arguments.
Replaces the 'build' step in CI with 'build-for-testing' and refactors the justfile to introduce reusable build and test recipes. Adds variables for derived data and build flags, and streamlines test invocation for specific targets. This improves maintainability and flexibility of build and test automation.
Introduces a new XCTest plan file for app unit tests and updates the Xcode project to include it in the resources build phase. This enables more flexible and configurable test execution for the GemTests target.
Removed unused 'xcbeautify' and 'list' recipes for clarity. Added '@' to several commands for consistent output suppression and improved the 'download-wallet-core' recipe to use silent curl.
Sets RUSTC_WRAPPER in the sccache setup step for GitHub-hosted runners instead of globally in ci.yml and ui_tests.yml. This ensures the environment variable is only set when appropriate.
Introduces a new 'test-ui-without-building' Justfile recipe and updates the UI test workflow to use it. This allows running UI tests without rebuilding, improving CI efficiency. Also adds a separate build-for-testing step to the workflow.
Replaces the commented-out 'rm -rf build/DerivedData' step with an active 'just clean' command in the CI workflow for improved consistency and maintainability.
@gemcoder21 gemcoder21 marked this pull request as ready for review January 24, 2026 22:24
@gemcoder21 gemcoder21 merged commit 6f58440 into main Jan 24, 2026
1 check passed
@gemcoder21 gemcoder21 deleted the ci-runner branch January 24, 2026 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant